Coverage decreased (-0.2%) to 97.877% when pulling 2b12b68 on KlaasJelmer:master into 833b375 on NaturalHistoryMuseum:master.

Is there a good way to get attention to this pull request?

This would be quite a nice and useful workaround for issues in Europe, where the base library zbar mis-interprets the encoding:

• https://github.com/mchehab/zbar/blob/9f0cbc70aba627afe02cb0afdb8fbc39eb4f229d/zbar/qrcode/qrdectxt.c#L212
• Wrong text encoding assumption: Wrong text encoding assumption mchehab/zbar#212
• Related to Decoding umlauts: Decoding umlauts #14

My workaround for now is the following, which could be avoided if one could just get the binary representation of the QR codes:

  try:
    logging.info("  - Trying the ugly workaround to re-encode utf-8 to big5, then back to utf-8")
    logging.info(res[0].data.decode('utf-8').encode('big5'))
    return res[0].data.decode('utf-8').encode('big5').decode('utf-8')
  except:
    logging.info("  - Re-encoding failed, so the qrcode is probably ok already in utf-8")
    logging.info(res[0].data.decode('utf-8'))
    return res[0].data.decode('utf-8')

Getting the same issue. Hope this pull request gets more attention as soon as possible!

I have the same problem and am having trouble. I strongly hope that this PR will get some attention.

Please could we get this PR merged into the master?
Or maybe an explanation what this is not possible at the moment?
Thank you!

Even with this PR, binary support is incomplete: null bytes are treated as string terminators.